Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Independence posets #37601

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from
Draft

Independence posets #37601

wants to merge 7 commits into from

Conversation

fchapoton
Copy link
Contributor

@fchapoton fchapoton commented Mar 12, 2024 •
edited
Loading

this is from #33659, with clean-up.

Note : related code can be found in https://github.com/haruhisa-enomoto/tors-lattice

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

@tscrim
Copy link
Collaborator

tscrim commented Mar 12, 2024

Still some more cleanup to do (e.g., _inc needs doctests, lots of one-line docstrings needed). I will try to do this soon. Although it seems like there is an additional class to be constructed.

Relevant paper: https://arxiv.org/abs/1805.00815

@fchapoton
Copy link
Contributor Author

yes, absolutely not ready, and parts of that are already in sage, for instance is_trim. I just wanted to start cleaning.

dcoudert
dcoudert suggested changes Mar 13, 2024
View reviewed changes
@fchapoton fchapoton force-pushed the independence_posets branch from cd0a5b6 to 18dcd4d Compare March 13, 2024 08:31
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 14, 2024 •
edited
Loading

Documentation preview for this PR (built with commit 43d97fc; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

dcoudert
dcoudert suggested changes Jun 17, 2024
View reviewed changes
@dcoudert
Copy link
Contributor

There is a lint error

@fchapoton fchapoton force-pushed the independence_posets branch from e22eb6b to 2870759 Compare June 22, 2024 05:40
@fchapoton fchapoton marked this pull request as draft July 10, 2024 15:17
@fchapoton
Copy link
Contributor Author

there are still some one-line descriptions missing. Also one may like to have more explicit function names

@fchapoton fchapoton force-pushed the independence_posets branch from 0f14f55 to 442a6b0 Compare July 11, 2024 08:50
@fchapoton fchapoton mentioned this pull request Jan 27, 2025
Open
1 task
@fchapoton fchapoton mentioned this pull request Feb 20, 2025
Open
5 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dcoudert dcoudert dcoudert requested changes

Assignees
No one assigned
Labels
c: combinatorics v: large
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@fchapoton @tscrim @dcoudert @mkoeppe @Vathys